Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Pango fixes #674

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[WIP] Pango fixes #674

wants to merge 3 commits into from

Conversation

certik
Copy link
Member

@certik certik commented Feb 23, 2015

No description provided.

@certik
Copy link
Member Author

certik commented Feb 23, 2015

As of a22f87c, it fails with:

[pango] checking if g++ supports -c -o file.o... yes
[pango] checking if g++ supports -c -o file.o... (cached) yes
[pango] checking whether the g++ linker (/usr/bin/ld -m elf_x86_64) supports shared libraries... yes
[pango] checking dynamic linker characteristics... (cached) GNU/Linux ld.so
[pango] checking how to hardcode library paths into programs... immediate
[pango] configure: creating ./config.lt
[pango] config.lt: creating libtool
[pango] checking for some Win32 platform... no
[pango] checking for pkg-config... /local/certik/bld/profile/bto56eplrk4v/bin/pkg-config
[pango] checking pkg-config is at least version 0.9.0... 
[pango] yes
[pango] checking for HARFBUZZ... no
[pango] 
[pango] no
[pango] checking for CoreText availability... no
[pango] checking for CAIRO... 
[pango] yes
[pango] checking which cairo font backends could be used... none
[pango] configure: Disabling cairo support
[pango] configure: error: *** Could not enable any backends.
[pango] *** Must have at least one backend to build Pango.
[pango|ERROR] Command '[u'/bin/bash', '_hashdist/build.sh']' returned non-zero exit status 1
[pango|ERROR] command failed (code=1); raising

The older version failed with missing glib.h library, even though it was specifically included.

@certik certik mentioned this pull request Feb 23, 2015
@jcftang
Copy link
Member

jcftang commented Feb 23, 2015

I think pango and cairo are circular dependencies.

@ahmadia
Copy link
Contributor

ahmadia commented Feb 23, 2015

Also, please pull the zlib/png updates into separate PRs. They affect the
entire stack since they're dependencies everywhere, and they are likelier
to be reverted if something goes bad.

On Mon, Feb 23, 2015 at 5:31 PM, Jimmy Tang [email protected]
wrote:

I think pango and cairo are circular dependencies.


Reply to this email directly or view it on GitHub
#674 (comment).

@certik
Copy link
Member Author

certik commented Feb 24, 2015

@ahmadia I've already done that in #671, this PR just depends on it. Once #671 is merged, I can rebase this PR.

@certik
Copy link
Member Author

certik commented Feb 24, 2015

@jcftang so we need to bootstrap cairo and pango using the same technique as gettext?

@ahmadia
Copy link
Contributor

ahmadia commented Feb 24, 2015

Great! Carry on then :)

@jcftang
Copy link
Member

jcftang commented Feb 24, 2015

@certik I think some experimenting might be needed to get the dependencies right, but yes the simple solution would to make it like what gettext does. Do you happen to have cairo already built in the stack that you are testing on right now?

@certik
Copy link
Member Author

certik commented Feb 24, 2015

Yes, I have cairo built in the stack, since it fails with pango, which requires cairo.

@ahmadia, #671 is merged and the #673 contains patches up to 7fcf4f2 from this PR (rebased though). After that's in, I'll rebase this PR.

Each of these must be investigated and seen if it is really needed.
@jcftang
Copy link
Member

jcftang commented Mar 13, 2015

I think if pango and cairo are updated in seperate commits, it is worth merging. It will probably break a few other GTK2 dependencies, but its worth upgrading those as they come along as well.

@cekees
Copy link
Contributor

cekees commented Oct 20, 2015

It looks to me like this can go in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants